Skip to content

feat(ui5-search): max-height for suggestions dropdown#13603

Open
aleksandar-terziev wants to merge 2 commits into
mainfrom
suggestion-dropdown-max-height
Open

feat(ui5-search): max-height for suggestions dropdown#13603
aleksandar-terziev wants to merge 2 commits into
mainfrom
suggestion-dropdown-max-height

Conversation

@aleksandar-terziev
Copy link
Copy Markdown
Contributor

@aleksandar-terziev aleksandar-terziev commented May 28, 2026

JIRA: BGSOFUIRILA-4504

  • expose search-popover to allow dynamic max-height for the search popover

  • default max-height is 39rem

JIRA: BGSOFUIRILA-4504

- expose `search-popover` to allow dynamic max-height for the search popover

- default max-height is 39rem
@aleksandar-terziev aleksandar-terziev requested a review from MapTo0 May 28, 2026 11:54
@ui5-webcomponents-bot
Copy link
Copy Markdown
Collaborator

ui5-webcomponents-bot commented May 28, 2026

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 28, 2026 12:01 Inactive
@ivoplashkov ivoplashkov requested a review from niyap June 2, 2026 07:37
* @public
* @since 2.9.0
* @experimental
* @csspart search-popover - Used to style the suggestions popover content
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would strip search from the name as the component is Search, so maybe popover, dropdown, or suggestion-popover

Copy link
Copy Markdown
Collaborator

@ilhan007 ilhan007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor comment regarding the part name, otherwise seems fine + 1

@ilhan007 ilhan007 temporarily deployed to netlify-preview June 4, 2026 12:03 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants